Projet Zuul: Exercice 7.35.1

Forum des exercices du projet Zuul

Exercice 7.35.1

Vous êtes abonné à cette discussion. Cliquez pour vous désabonner.

En s'inspirant de processCommand() de la classe Game dans zuul-with-enums-v1, remplacer dans interpreteCommand() de la classe GameEngine dans son jeu la suite de if else par un switch (autorisé sur un type énuméré !) .

Tout recompiler : il ne doit y avoir aucun warning !

Permalien | Modifier | Supprimer | Répondre

Peut-on utiliser la réflexion à la place du switch ?

Non, le switch est clairement la structure la plus appropriée pour gérer une variable de type enum.

L'API reflection de Java est une grosse machinerie peu performante, et il faudra y regarder à deux fois si vous trouvez une fonctionnalité du jeu pour laquelle elle pourrait avoir un intérêt.

Bonjour,

Après avoir ecrit le switch, le compilateur nous indique un message d'erreur : " an enum switch case label must be the unqualified name of a enumeration constant"

CommandWord commandWord = command.getCommandWord();

        switch(commandWord){
            case CommandWord.aide :
                aide();
                break;

L'expression à tester et la valeur sont bien des commandWord.

Un 'qualified name' est de la forme Classe.membre alors qu'un 'unqualified name' est simplement de la forme membre, et un 'case label' est ce qui se trouve entre case et :.

D'autre part, les constantes d'un enum sont toujours écrites en majuscules (regardez l'exemple dans zuul-with-enum).

Bonjour,

j'ai un problème qui concerne le switch.

Dans interpretCommand j'ai :

aGui.println(pCommandLine);
Command vCommand = aParser.getCommand(pCommandLine);
CommandWord vCommandWord = vCommand.getCommandWord();

switch(vCommandWord)
{
.../... code supprimé .../...

case UNKNOWN : aGui.println("Cette commande n'existe pas");
break;
.../... code supprimé .../...
}

Mais quand je lance le jeu, quelque soit les commandes que je rentre, ça m'affiche "Cette commande n'existe pas", donc la commande est inconnue.

Je pense que le pb ne vient pas du code ci-dessus, mais de l'endroit où votre programme décide de choisir le CommandWord UNKNOWN en fonction de la String que vous avez tapée. Les String dans CommandWords sont-elles correctes ?

Il me semble qu'elles le sont, j'ai comme constructeur par défault de la classe CommandWords :

public CommandWords()
{
this.aValidCommands = new HashMap<String, CommandWord>();
this.aValidCommands.put("go", CommandWord.GO);
.../... code supprimé .../...
}

Ce qui permet d'initialiser la HashMap aValidCommands.

Et pour reconnaître la commande tapé :

public CommandWord getCommandWord(final String pCommandWord)
{
CommandWord vCommand = this.aValidCommands.get(pCommandWord);
if(vCommand != null)

{
return vCommand;
}
else

{
return CommandWord.UNKNOWN;
}
}

Qui doit normalement retourner la commande UNKNOWN si la commande est inconnue et retourner la commande passé en paramètre si elle est connue.

Si getCommandWord retourne UNKNOWN, c'est que vCommand vaut null, donc que la String pCommandWord n'a pas été trouvée dans la HashMap.
Ajoutez au début de la méthode getCommandWord un affichage de pCommandWord, et s'il est correct, un affichage de this.aValidCommands qui vous montrera toutes les associations stockées dans la HashMap.
Cela devrait vous aider à comprendre d'où vient le problème.

Revenir à Généralités

Projet Zuul de conception orientée objet en Java d'un jeu d'aventure

Forum des exercices du projet Zuul

Exercice 7.35.1

Navigation

Administration

Navigation

Administration